Skip to content

Fix Azure Login in pr-check - #171

Merged
zijchen merged 3 commits into
masterfrom
zijchen/fixlogin
Mar 10, 2023
Merged

Fix Azure Login in pr-check#171
zijchen merged 3 commits into
masterfrom
zijchen/fixlogin

Conversation

@zijchen

@zijchen zijchen commented Mar 10, 2023

Copy link
Copy Markdown
Contributor

Use OIDC federated credentials
Also refreshed the service principal secrets since they are needed for the service principal deployment scenarios

Fixes #169

@zijchen
zijchen temporarily deployed to Automation test March 10, 2023 19:10 — with GitHub Actions Inactive
@zijchen
zijchen temporarily deployed to Automation test March 10, 2023 19:10 — with GitHub Actions Inactive
@zijchen
zijchen temporarily deployed to Automation test March 10, 2023 19:10 — with GitHub Actions Inactive
@zijchen
zijchen temporarily deployed to Automation test March 10, 2023 19:10 — with GitHub Actions Inactive
@zijchen
zijchen temporarily deployed to Automation test March 10, 2023 19:10 — with GitHub Actions Inactive
@zijchen
zijchen temporarily deployed to Automation test March 10, 2023 19:10 — with GitHub Actions Inactive
@zijchen
zijchen marked this pull request as ready for review March 10, 2023 19:17
@zijchen
zijchen requested review from dzsquared and yorek as code owners March 10, 2023 19:17
@zijchen
zijchen requested a review from chlafreniere March 10, 2023 19:18
@zijchen

zijchen commented Mar 10, 2023

Copy link
Copy Markdown
Contributor Author

pr-check is green now

@zijchen
zijchen merged commit 36c52fa into master Mar 10, 2023
@zijchen
zijchen deleted the zijchen/fixlogin branch March 10, 2023 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pipeline credentials expired

2 participants